-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
✨ [#108] Admin action to check Abonnement callback status #186
base: main
Are you sure you want to change the base?
✨ [#108] Admin action to check Abonnement callback status #186
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, works for me
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It works for me, but I'm afraid that if there are a lot of subscribers the admin page would be loading for a while
Perhaps we should only show it in the detail view?
What do you think?
@annashamray hmm that's right, though if it's on the detail view, it isn't really convenient if you want to check multiple subscriptions, because you'd have to open every detail page. Maybe there should be a button/action to trigger it? Also come to think of it, I'm not sure if the |
7f4e311
to
401b44b
Compare
with a custom button to run this action for all Abonnementen
401b44b
to
2238c16
Compare
2238c16
to
223bfde
Compare
@annashamray I've changed it to a custom admin action with a shortcut button to trigger it for all Subscriptions. I've also added a filter on the result to make it easier to check which status checks failed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yep, this too works for me
fixes #108
Changes: